Skip to content
This repository has been archived by the owner on May 10, 2019. It is now read-only.

106: Add Slider and SliderInput components #124

Merged

Conversation

Caleb-Ellis
Copy link
Contributor

Done

  • Added Slider component and SliderInput subcomponent
  • Added Storybook stories for Slider
  • Added basic snapshot and enzyme tests for Slider
  • Added a simple ensureValueInRange.js utility function. I'll combine this with getClassName.js in a single utils.js file in a separate PR.

QA

  • Check out this feature branch
  • Run the site using the command ./run serve
  • View the site locally in your web browser at: http://0.0.0.0:8102/
  • Check that the Slider component operates as you would expect it to according to the Vanilla docs
  • Play around with the knobs panel and see how you can interact with the Slider component, and whether the props API works nicely

Issue / Card

Fixes #106

@webteam-app
Copy link

@steverydz
Copy link
Contributor

@Caleb-Ellis Looks good to me but the percy check is failing. I don't have access so can't see why?

@kwm14 kwm14 merged commit 7adf4c9 into canonical-web-and-design:master Mar 23, 2018
@Caleb-Ellis
Copy link
Contributor Author

@steverydz I think we need @pmahnke or @barrymcgee to give you access to Percy - we'll get it done shortly. And thanks!

@Caleb-Ellis Caleb-Ellis deleted the 106-slider-component branch March 23, 2018 12:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants